Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating presets #9642

Merged
merged 13 commits into from
Sep 22, 2021
Merged

Updating presets #9642

merged 13 commits into from
Sep 22, 2021

Conversation

Tomertech
Copy link
Contributor

@Tomertech Tomertech commented Aug 17, 2021

Updated hard-coded presets.
Added a D450 preset.
Updated the function that chooses the correct presets.

Tested by comparing the JSONs files of the new presets (in the ticket) to the exported JSONs files from the viewer.

Edit: I have added a fix for the inaccuracy of setting an int field in the SDK with a float value from the json preset.
Edit2: added two resolutions based presets to D455 as solutions for DSO-17597.

Tracked on DSO-17183.

Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, waiting for a confirmation with IQ before merge

src/ds5/advanced_mode/presets.cpp Show resolved Hide resolved
default_450(p);
break;
//case ds::RS455_PID:
case ds::RS435I_PID:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D455

src/ds5/advanced_mode/presets.cpp Show resolved Hide resolved
@ev-mp ev-mp marked this pull request as ready for review September 13, 2021 06:29
src/ds5/advanced_mode/presets.cpp Show resolved Hide resolved
src/ds5/advanced_mode/presets.cpp Outdated Show resolved Hide resolved
src/ds5/advanced_mode/presets.cpp Outdated Show resolved Hide resolved
src/ds5/advanced_mode/presets.cpp Show resolved Hide resolved
@ev-mp
Copy link
Collaborator

ev-mp commented Sep 19, 2021

Go over MSVC warnings

Copy link
Collaborator

@ev-mp ev-mp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ev-mp ev-mp merged commit 6d79d80 into IntelRealSense:development Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants